Skip to content

Add a reference links to incident integrations #29480

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 8 commits into from
May 30, 2025
Merged

Add a reference links to incident integrations #29480

merged 8 commits into from
May 30, 2025

Conversation

estherk15
Copy link
Contributor

@estherk15 estherk15 commented May 21, 2025 •
edited by jira bot
Loading

What does this PR do? What is the motivation?

  • Add reference links to Incident management integrations
  • Remove the Private Zoom page since the integration page is public and provides the same detail of information.
  • DOCS-5932

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

To have your PR automatically merged after it receives the required reviews, add the following PR comment:

/merge

Additional notes

@estherk15 estherk15 requested a review from a team as a code owner May 21, 2025 20:09
@github-actions GitHub Actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Removed or renamed files (these should redirect)

Modified Files

buraizu
buraizu reviewed May 21, 2025
View reviewed changes
Copy link
Contributor

@buraizu buraizu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left one suggestion, but also wanted to confirm if it's expected for the service_management/incident_management/zoom_integration link to not be redirecting. Would it potentially be helpful to add a redirect for this in the pull_config.yaml file, for anyone who might have this link bookmarked?

@estherk15
Copy link
Contributor Author

Left one suggestion, but also wanted to confirm if it's expected for the service_management/incident_management/zoom_integration link to not be redirecting. Would it potentially be helpful to add a redirect for this in the pull_config.yaml file, for anyone who might have this link bookmarked?

Good call, I assumed it wasn't used, but you never know!

@estherk15 estherk15 requested a review from a team as a code owner May 23, 2025 16:29
@github-actions github-actions bot added the Architecture Everything related to the Doc backend label May 23, 2025
@estherk15 estherk15 requested a review from buraizu May 23, 2025 20:38
estherk15
estherk15 commented May 27, 2025
View reviewed changes
buraizu
buraizu reviewed May 28, 2025
View reviewed changes
Copy link
Contributor

@buraizu buraizu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for following up @estherk15, but the redirect doesn't seem to be working. I've left a suggestion that seemed to work in a local build.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 28, 2025 •
edited
Loading

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

@renteria1010 renteria1010 self-requested a review May 29, 2025 21:39
@estherk15 estherk15 merged commit 02480ba into master May 30, 2025
14 checks passed
@estherk15 estherk15 deleted the esther/docs-5932-zoom-incident-management branch May 30, 2025 14:20
estherk15 added a commit that referenced this pull request May 30, 2025
@estherk15
Revert "Add a reference links to incident integrations (#29480)"
cd0e0c9 
This reverts commit 02480ba.
@estherk15 estherk15 mentioned this pull request May 30, 2025
Merged
brett0000FF pushed a commit that referenced this pull request May 30, 2025
@estherk15
Revert "Add a reference links to incident integrations (#29480)" (#29638
0cfe244 
)

This reverts commit 02480ba.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@buraizu buraizu buraizu approved these changes

@renteria1010 renteria1010 renteria1010 approved these changes

Assignees
No one assigned
Labels
Architecture Everything related to the Doc backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@estherk15 @buraizu @renteria1010